Conversation
prajwal-pai77
commented
Apr 17, 2026
prajwal-pai77
commented
- Add google analytics
- Add cookie banner
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the Docusaurus configuration to include OneTrust cookie consent scripts and tracking via Google Tag Manager and Google Analytics. Feedback was provided regarding a potential domain mismatch for the OneTrust script ID, the redundancy of using both GTM and gtag simultaneously which can cause duplicate tracking, and the unnecessary anonymizeIP setting for GA4 properties.
| googleTagManager: { | ||
| containerId: 'GTM-MVPNN2', | ||
| }, | ||
| gtag: { | ||
| trackingID: 'G-CVKKEY0D6B', | ||
| anonymizeIP: true, | ||
| }, |
There was a problem hiding this comment.
Configuring both googleTagManager and gtag simultaneously is generally discouraged. This setup results in both tracking scripts being loaded, which can lead to duplicate event tracking (such as double page views) and negatively impact site performance. You should choose one tracking method. If you prefer GTM, you can manage your GA4 property within the GTM container and remove the gtag block.
| gtag: { | ||
| trackingID: 'G-CVKKEY0D6B', | ||
| anonymizeIP: true, | ||
| }, |
There was a problem hiding this comment.
The anonymizeIP option is unnecessary for Google Analytics 4 (GA4) properties (IDs starting with G-). GA4 anonymizes IP addresses by default, and this setting is ignored by the script.
gtag: {
trackingID: 'G-CVKKEY0D6B',
},1 participant
